Skip to content

add basic test of IDOM #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jul 21, 2021
Merged

Conversation

rmorshea
Copy link

No description provided.

@rmorshea
Copy link
Author

@Archmonger can you approave/merge this?

@Archmonger Archmonger self-requested a review July 20, 2021 18:33
@Archmonger
Copy link
Owner

@Archmonger can you approave/merge this?

Awaiting one response on a potential typo. Let me know if it was intentional.

@rmorshea
Copy link
Author

I think you forgot to publish the review.

@@ -32,4 +32,4 @@ jobs:
run: |
npm install -g npm@latest
npm --version
nox -s test
nox -s test -- --headless
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was the double dash (-- --headless)intentional here?

Copy link
Author

@rmorshea rmorshea Jul 21, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah that was intentional - it delimits CLI args for nox vs args that end up a Session.posargs in noxfile.py (see here)

@Archmonger
Copy link
Owner

I think you forgot to publish the review.

My bad, published.

@Archmonger Archmonger dismissed their stale review July 21, 2021 04:37

Resolved

@Archmonger Archmonger merged commit 4d440eb into Archmonger:main Jul 21, 2021
Archmonger pushed a commit that referenced this pull request Sep 7, 2021
Make django_idom an installable app
@Archmonger Archmonger deleted the archmonger branch January 21, 2022 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants